Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ensures $xml_items is always an array before it's treated as such. #3

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

CoalaWeb
Copy link

Fixes PHP 7.1 warnings regarding “Illegal string offset” and “Cannot assign an empty string”.

Fixes PHP 7.1 warnings regarding “Illegal string offset” and “Cannot assign an empty string”.
@CoalaWeb
Copy link
Author

Is there a reason that you released a version 9 days after this pull request with out the fix?

@PhocaCz
Copy link
Owner

PhocaCz commented Jun 30, 2017

Hi, not sure what you exactly mean, the latest version: 3.0.3 includes this fix:

You can download it here:
https://www.phoca.cz/download/category/96-phoca-commander-component

News:
https://www.phoca.cz/news/942-phoca-commander-3-0-3-released

@CoalaWeb
Copy link
Author

CoalaWeb commented Jul 1, 2017

Great! I was just a bit confused because this pull request hasn't been merged and the code in the main branch doesn't appear to have the changes. The main branch appears to be from two years ago (v3.00). Cheers and I think PhocaCommander is an awesome extension ;)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants